Skip to content

fix: secure knowledge assistant routes - #4585

Merged
Ayushh-Sharmaa merged 1 commit into
Ayushh-Sharmaa:mainfrom
AMAN194701:fix/4565-knowledge-assistant-auth-rate-limit
Aug 12, 2026
Merged

fix: secure knowledge assistant routes#4585
Ayushh-Sharmaa merged 1 commit into
Ayushh-Sharmaa:mainfrom
AMAN194701:fix/4565-knowledge-assistant-auth-rate-limit

Conversation

@AMAN194701

Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes #4565

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Tested locally
  • Checked for console warnings and errors

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have checked my code and corrected any misspellings

Signed-off-by: Aman Kushwaha <amankushwaha61709@gmail.com>
@vercel

vercel Bot commented Aug 9, 2026

Copy link
Copy Markdown

@AMAN194701 is attempting to deploy a commit to the ayushh-sharmaa's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added backend Backend / server related size/XS Extra-small PR (< 10 lines) labels Aug 9, 2026

const knowledgeAssistantController = require("../controllers/knowledgeAssistantController");

router.use(requireStudentAuth);
@github-actions github-actions Bot added backlog size gssoc gssoc approved level GSSoC'26 Automatically applied to issues for GSSoC'26 NSOC'26 Automatically applied to newly opened issues for NSOC'26 labels Aug 11, 2026
@Ayushh-Sharmaa
Ayushh-Sharmaa merged commit 506a5a0 into Ayushh-Sharmaa:main Aug 12, 2026
11 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend / server related backlog size gssoc approved gssoc GSSoC'26 Automatically applied to issues for GSSoC'26 level NSOC'26 Automatically applied to newly opened issues for NSOC'26 size/XS Extra-small PR (< 10 lines)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Unauthenticated Access and Lack of Rate Limiting on Natural Language Search Assistant

3 participants